Skip to content

v3.20.1#1453

Merged
danielroe merged 1 commit intomainfrom
v3.20.1
Oct 24, 2025
Merged

v3.20.1#1453
danielroe merged 1 commit intomainfrom
v3.20.1

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

3.20.1 is the next patch release.

Timetable: to be announced.

👉 Changelog

compare changes

🏡 Chore

  • Move shell-emulator config to pnpm-workspace file (6860f0faf)

🤖 CI

  • Use pnpm publish and remove registry url (b46b6b16d)

❤️ Contributors

@danielroe danielroe marked this pull request as ready for review October 24, 2025 08:31
@danielroe danielroe self-requested a review as a code owner October 24, 2025 08:31
@danielroe danielroe merged commit 34aabfb into main Oct 24, 2025
@danielroe danielroe deleted the v3.20.1 branch October 24, 2025 08:31
@Yizack
Copy link
Copy Markdown
Contributor

Yizack commented Oct 24, 2025

Hi, for the release workflow this should work

  steps:
    - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
      with:
        fetch-depth: 0
    - run: corepack enable
    - uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
      with:
        node-version: latest
        cache: pnpm
+       registry-url: https://registry.npmjs.org
+
+   - run: pnpm config set registry https://registry.npmjs.org

    - name: 📦 Install dependencies
      run: pnpm install

    - name: 🛠 Prepare build environment
      run: pnpm dev:prepare

    - name: 📦 Release
+     run: pnpm publish --access public --no-git-checks

    - name: 🏷️ Create tag
      run: |
        TAG_NAME=${{ github.event.pull_request.head.ref }}
        git tag $TAG_NAME
        git push origin $TAG_NAME

@danielroe
Copy link
Copy Markdown
Member

what I can't figure out is why it isn't working in this repo but working fine in nuxt/cli + nuxt/nuxt, etc.

@Yizack
Copy link
Copy Markdown
Contributor

Yizack commented Oct 24, 2025

From what I see, nuxt/nuxt uses current node version and nuxt/cli uses npm instead of pnpm to publish

There's a weird behaviour when using latest node version and pnpm to publish with OIDC, not sure the reason but it is needed to do pnpm config set registry https://registry.npmjs.org too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants